Skip to content

Conversation

@winstonzhang-intel
Copy link
Contributor

@winstonzhang-intel winstonzhang-intel commented Sep 26, 2024

LLVM/SYCL test: intel/llvm#15588

@github-actions github-actions bot added the level-zero L0 adapter specific issues label Sep 26, 2024
@winstonzhang-intel winstonzhang-intel marked this pull request as ready for review October 3, 2024 01:25
@winstonzhang-intel winstonzhang-intel requested a review from a team as a code owner October 3, 2024 01:25
Copy link
Contributor

@nrspruit nrspruit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please address my comment.

@winstonzhang-intel winstonzhang-intel force-pushed the counter-based-3 branch 2 times, most recently from dc26d7d to 9b67cb7 Compare October 9, 2024 14:39
@nrspruit
Copy link
Contributor

E2E failures are unrelated:


Testing: 0.. 10.. 20.. 30.. 40.. 50.. 60.. 70.. 80.. 90..


Failed Tests (4):
SYCL :: ESIMD/ext_math.cpp
SYCL :: ESIMD/ext_math_fast.cpp
SYCL :: ESIMD/ext_math_ieee_sqrt_div.cpp
SYCL :: ESIMD/ext_math_saturate.cpp

These are failing due to:

| --- TESTING ESIMD rsqrt, T=d, N = 32...

| rsqrt test, kind=ESIMD...

| *** ERROR. SYCL exception caught: << The program was built for 1 devices

| Build program log for 'Intel(R) Data Center GPU Max 1100':

| Module <0x2d0cb80>: Unresolved Symbol <_Z17__spirv_ocl_rsqrtDv32_d>

| Module <0x2d0cb80>: Unresolved Symbol <_Z17__spirv_ocl_rsqrtDv32_d>

Which is not related to the changes here where modules are unchanged.

winstonzhang-intel added a commit to winstonzhang-intel/llvm that referenced this pull request Oct 10, 2024
winstonzhang-intel added a commit to winstonzhang-intel/llvm that referenced this pull request Oct 10, 2024
winstonzhang-intel added a commit to winstonzhang-intel/llvm that referenced this pull request Oct 10, 2024
winstonzhang-intel added a commit to winstonzhang-intel/llvm that referenced this pull request Oct 10, 2024
winstonzhang-intel added a commit to winstonzhang-intel/llvm that referenced this pull request Oct 10, 2024
@nrspruit nrspruit added the ready to merge Added to PR's which are ready to merge label Oct 11, 2024
@aarongreig aarongreig merged commit 4f8aa0d into oneapi-src:main Oct 15, 2024
77 of 78 checks passed
sarnex pushed a commit to intel/llvm that referenced this pull request Oct 15, 2024
PR in UR: oneapi-src/unified-runtime#2138

---------

Signed-off-by: Zhang, Winston <[email protected]>
Signed-off-by: Winston Zhang <[email protected]>
Co-authored-by: Aaron Greig <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

level-zero L0 adapter specific issues ready to merge Added to PR's which are ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants